Skip to content

Fix empty array edge cases in mvregress - #486

Merged
pr0m1th3as merged 4 commits into
gnu-octave:mainfrom
Sonu0305:fix-mvregress-empty
Sep 16, 2026
Merged

pr0m1th3as merged 4 commits into
gnu-octave:mainfrom
Sonu0305:fix-mvregress-empty

Conversation

@Sonu0305

Copy link
Copy Markdown
Contributor

No description provided.

@pr0m1th3as

Copy link
Copy Markdown
Member

Message matches MATLAB exactly, no need to replicate exact wording. Remove the %!test block that holds only a comment and keep the two %!error tests with the other input validation tests.

@Sonu0305

Copy link
Copy Markdown
Contributor Author

updated @pr0m1th3as
can you review again, thank you.

@pr0m1th3as

Copy link
Copy Markdown
Member

Keep the case of argument names: mvregress: Y cannot be an empty array. Please undo the reformatting of the two unrelated algorithm and bogus tests, and write zeros (0, 3) with spaces.

@Sonu0305

Copy link
Copy Markdown
Contributor Author

updated @pr0m1th3as

@pr0m1th3as
pr0m1th3as merged commit 03f81c7 into gnu-octave:main Sep 16, 2026
@Sonu0305
Sonu0305 deleted the fix-mvregress-empty branch September 16, 2026 03:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants